Skip to content

New Package: Google Cloud SDK#4528

Closed
aaronlevin wants to merge 6 commits intoNixOS:masterfrom
aaronlevin:google-cloud-sdk
Closed

New Package: Google Cloud SDK#4528
aaronlevin wants to merge 6 commits intoNixOS:masterfrom
aaronlevin:google-cloud-sdk

Conversation

@aaronlevin
Copy link
Contributor

@aaronlevin aaronlevin commented Oct 14, 2014

This is my first non-Haskell package. Google manages this package strangely: it's not version'd, uses $HOME during installation, etc. Hopefully I did the right things (I tried to look at other similar packages for inspiration).

edolstra
edolstra reviewed Oct 15, 2014
View reviewed changes
pkgs/tools/virtualization/google-cloud-sdk/default.nix Outdated
Copy link
Member

@edolstra edolstra Oct 15, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Descriptions should be very concise, e.g. Tools and libraries for managing Google Cloud Platform resources.

@aaronlevin
Copy link
Contributor Author

aaronlevin commented Oct 15, 2014

@edolstra I made the updates. Looking into why Travis CI is reporting a failure (something about python 3.4 which is odd because I specify python 2.7). Currently compiling locally but it's doing the jdk from scratch so it'll take a bit.

@aaronlevin
Copy link
Contributor Author

aaronlevin commented Oct 16, 2014

@edolstra this seems to be building now.

@aristidb
Copy link
Contributor

aristidb commented Oct 22, 2014

After fixing the SHA256:

Traceback (most recent call last):
  File "bin/bootstrapping/install.py", line 368, in <module>
    main()
  File "bin/bootstrapping/install.py", line 347, in main
    pargs.additional_components)
  File "bin/bootstrapping/install.py", line 310, in Install
    components = InstallComponents(to_install)
  File "bin/bootstrapping/install.py", line 332, in InstallComponents
    components.update(component_ids=component_ids, allow_no_backup=True)
  File "/tmp/nix-build-google-cloud-sdk.drv-0/google-cloud-sdk/bin/bootstrapping/../../lib/googlecloudsdk/calliope/frontend.py", line 235, in __call__
    post_run_hooks=None, kwargs=kwargs)
  File "/tmp/nix-build-google-cloud-sdk.drv-0/google-cloud-sdk/bin/bootstrapping/../../lib/googlecloudsdk/calliope/frontend.py", line 274, in _Execute
    pre_run_hooks=pre_run_hooks, post_run_hooks=post_run_hooks)
  File "/tmp/nix-build-google-cloud-sdk.drv-0/google-cloud-sdk/bin/bootstrapping/../../lib/googlecloudsdk/calliope/backend.py", line 882, in Run
    result = command_instance.Run(args)
  File "/tmp/nix-build-google-cloud-sdk.drv-0/google-cloud-sdk/lib/googlecloudsdk/gcloud/sdktools/components/update.py", line 65, in Run
    args.component_ids, allow_no_backup=args.allow_no_backup)
  File "/tmp/nix-build-google-cloud-sdk.drv-0/google-cloud-sdk/bin/bootstrapping/../../lib/googlecloudsdk/core/updater/update_manager.py", line 407, in Update
    install_state, diff = self._GetStateAndDiff()
  File "/tmp/nix-build-google-cloud-sdk.drv-0/google-cloud-sdk/bin/bootstrapping/../../lib/googlecloudsdk/core/updater/update_manager.py", line 211, in _GetStateAndDiff
    latest_snapshot = self._GetLatestSnapshot()
  File "/tmp/nix-build-google-cloud-sdk.drv-0/google-cloud-sdk/bin/bootstrapping/../../lib/googlecloudsdk/core/updater/update_manager.py", line 207, in _GetLatestSnapshot
    return snapshots.ComponentSnapshot.FromURLs(*self.__url.split(','))
  File "/tmp/nix-build-google-cloud-sdk.drv-0/google-cloud-sdk/bin/bootstrapping/../../lib/googlecloudsdk/core/updater/snapshots.py", line 123, in FromURLs
    for url in urls]
  File "/tmp/nix-build-google-cloud-sdk.drv-0/google-cloud-sdk/bin/bootstrapping/../../lib/googlecloudsdk/core/updater/snapshots.py", line 153, in _DictFromURL
    raise URLFetchError(extra_repo=extra_repo)
googlecloudsdk.core.updater.snapshots.URLFetchError: Failed to fetch component listing from server. Check your network settings and try again.

@aaronlevin
Copy link
Contributor Author

aaronlevin commented Oct 23, 2014

@aristidb could be a regression on google's part? I will remove the sha check and retry.

@aaronlevin
Copy link
Contributor Author

aaronlevin commented Oct 23, 2014

@aristidb I've updated SHA. How do we manage artifacts whose SHAs might change?

@aaronlevin
Copy link
Contributor Author

aaronlevin commented Oct 23, 2014

@aristidb I just rebased onto HEAD and ran nix-build -A google-cloud-sdk without any issue.

@aristidb
Copy link
Contributor

aristidb commented Oct 25, 2014

There's some discussion about how to deal with such artifacts in #4542

I'm afraid we don't really have a good solution.

@aristidb
Copy link
Contributor

aristidb commented Oct 25, 2014

No, this is not a regression. I still have the same problem.

You probably don't use chroot builds, so network accessing during build time work. They're highly impure though.

I can't add packages that don't build in chroot.

@aristidb aristidb added the 9.needs: reporter feedback This issue needs the person who filed it to respond label Oct 25, 2014
@aaronlevin
Copy link
Contributor Author

aaronlevin commented Oct 26, 2014

@aristidb I will look into the issues wrt to chroot. Any specific instructions to force it to build with chroot? (I've not looked too deeply into how exactly chroot works).

@7c6f434c
Copy link
Member

7c6f434c commented Nov 15, 2014

Usually pre-downloading the things that the package tries to download works. Julia programming language is an example, I think.

@7c6f434c
Copy link
Member

7c6f434c commented Jan 18, 2015

@aaronlevin Do I understand it right that this currently works only without chroot builds enabled?

@aaronlevin
Copy link
Contributor Author

aaronlevin commented Jan 18, 2015

@7c6f434c It was working for me until I performed an upgrade and then receive a similar error. So, perhaps we should close this PR without merging.

@domenkozar
Copy link
Member

domenkozar commented Jan 27, 2015

@aaronlevin cool, let us know if you want to pursue the issue or close PR

@aaronlevin
Copy link
Contributor Author

aaronlevin commented Jan 29, 2015

@iElectric I will close this PR and submit a new one if I have time to figure out the nuances with this package. Thanks!

@aaronlevin aaronlevin closed this Jan 29, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

9.needs: reporter feedback This issue needs the person who filed it to respond

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@aaronlevin @aristidb @7c6f434c @domenkozar @edolstra @jgeerds